Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

travis: Enable rust-analysis package for more targets #41267

Merged
merged 1 commit into from
Apr 13, 2017

Conversation

alexcrichton
Copy link
Member

This commit enables the rust-analysis package to be produced for all targets
that are part of the dist-* suite of docker images on Travis. Currently
these packages are showing up with available = false in the
channel-rust-nightly.toml manifest where we'd prefer to have them show up for
all targets.

Unfortunately rustup isn't handling the available = false section well right
now, so this should also inadvertently fix the nightly regression.

This commit enables the `rust-analysis` package to be produced for all targets
that are part of the `dist-*` suite of docker images on Travis. Currently
these packages are showing up with `available = false` in the
`channel-rust-nightly.toml` manifest where we'd prefer to have them show up for
all targets.

Unfortunately rustup isn't handling the `available = false` section well right
now, so this should also inadvertently fix the nightly regression.
@rust-highfive
Copy link
Collaborator

r? @aturon

(rust_highfive has picked a reviewer for you, use r? to override)

@alexcrichton
Copy link
Member Author

@bors: r+ p=10

fixing a nightly regression in rust-lang/rustup#1062

@bors
Copy link
Contributor

bors commented Apr 13, 2017

📌 Commit cdedecb has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Apr 13, 2017

⌛ Testing commit cdedecb with merge 7bdd8fb...

@bors
Copy link
Contributor

bors commented Apr 13, 2017

💔 Test failed - status-appveyor

@arielb1
Copy link
Contributor

arielb1 commented Apr 13, 2017

@bors
Copy link
Contributor

bors commented Apr 13, 2017

⌛ Testing commit cdedecb with merge 1f59c7e...

bors added a commit that referenced this pull request Apr 13, 2017
travis: Enable rust-analysis package for more targets

This commit enables the `rust-analysis` package to be produced for all targets
that are part of the `dist-*` suite of docker images on Travis. Currently
these packages are showing up with `available = false` in the
`channel-rust-nightly.toml` manifest where we'd prefer to have them show up for
all targets.

Unfortunately rustup isn't handling the `available = false` section well right
now, so this should also inadvertently fix the nightly regression.
@bors
Copy link
Contributor

bors commented Apr 13, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 1f59c7e to master...

@bors bors merged commit cdedecb into rust-lang:master Apr 13, 2017
@alexcrichton alexcrichton deleted the fix-analysis-dist branch April 19, 2017 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants